Skip to content
This repository has been archived by the owner on Nov 23, 2024. It is now read-only.

docs: remove type hints #125

Merged
merged 5 commits into from
Jan 25, 2024
Merged

docs: remove type hints #125

merged 5 commits into from
Jan 25, 2024

Conversation

lars-reimann
Copy link
Member

Summary of Changes

Remove type hints in docstrings. These are redundant, since types are already specified in the code.

@lars-reimann lars-reimann requested a review from a team as a code owner January 25, 2024 11:50
Copy link
Contributor

github-actions bot commented Jan 25, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ MARKDOWN markdown-link-check 1 0 1.32s
✅ PYTHON black 3 0 0 0.53s
✅ PYTHON mypy 3 0 1.37s
✅ PYTHON ruff 3 0 0 0.04s
✅ REPOSITORY git_diff yes no 0.05s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

Copy link

codecov bot commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (df74160) 100.00% compared to head (8227754) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #125   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines           38        38           
=========================================
  Hits            38        38           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lars-reimann lars-reimann merged commit ca21f09 into main Jan 25, 2024
7 checks passed
@lars-reimann lars-reimann deleted the remove-type-hints-docs branch January 25, 2024 11:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants